-
Notifications
You must be signed in to change notification settings - Fork 1.6k
chore: librarian generate pull request: 20260210T151820Z #15553
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Summary of ChangesHello @parthea, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request significantly enhances the GKE Hub API by introducing a new set of resources and associated operations. The changes enable more granular management of GKE Fleets, including the ability to define and control Scopes, Namespaces, Membership Bindings, and RBAC Role Bindings. This expansion provides users with greater flexibility and control over their GKE environments through programmatic interfaces. Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
This pull request primarily updates the google-cloud-gke-hub library by regenerating its client code, as indicated by the last_generated_commit change in .librarian/state.yaml. The core changes involve introducing extensive new functionality related to GKE Hub's Fleet, Scope, Namespace, MembershipBinding, and RBACRoleBinding resources. This includes adding new type definitions for these resources and their lifecycle states in google/cloud/gkehub_v1/types/fleet.py, and updating google/cloud/gkehub/__init__.py and google/cloud/gkehub_v1/__init__.py to import and expose these new types and their associated service request/response messages. The gapic_metadata.json file is updated to reflect the new GAPIC client methods for creating, getting, updating, deleting, and listing these resources, as well as generating RBAC YAML. Correspondingly, new pager classes are added in pagers.py for paginated list operations, and the base.py, grpc.py, and grpc_asyncio.py transport implementations are extended to support the new methods. Additionally, rest_base.py gains new classes for REST API transcoding of these operations. The feature.py file is updated to include ScopeFeatureSpec and ScopeFeatureState, and to change terminology from 'Hub' to 'Fleet' in comments, along with adding rbacrolebindingactuation to CommonFeatureState and new fields for scope_specs, scope_states, and unreachable to the Feature message. The review comment suggests alphabetically sorting the symbols in the __all__ list for better maintainability and readability.
| "FeatureState", | ||
| "CommonFeatureSpec", | ||
| "CommonFeatureState", | ||
| "ScopeFeatureSpec", | ||
| "ScopeFeatureState", | ||
| "MembershipFeatureSpec", | ||
| "MembershipFeatureState", | ||
| }, | ||
| ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR created by the Librarian CLI to generate Cloud Client Libraries code from protos.
BEGIN_COMMIT
BEGIN_NESTED_COMMIT
docs: A comment for field
request_idin message.google.cloud.gkehub.v1.CreateFeatureRequestis changedPiperOrigin-RevId: 865111649
Library-IDs: google-cloud-gke-hub
Source-link: googleapis/googleapis@7a570661
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
docs: A comment for field
cancel_requestedin message.google.cloud.gkehub.v1.OperationMetadatais changedPiperOrigin-RevId: 865111649
Library-IDs: google-cloud-gke-hub
Source-link: googleapis/googleapis@7a570661
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
feat: Add GKE Hub teams scope and RBAC GAPIC
PiperOrigin-RevId: 865111649
Library-IDs: google-cloud-gke-hub
Source-link: googleapis/googleapis@7a570661
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
docs: A comment for message
Featureis changedPiperOrigin-RevId: 865111649
Library-IDs: google-cloud-gke-hub
Source-link: googleapis/googleapis@7a570661
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
docs: A comment for field
statein message.google.cloud.gkehub.v1.Featureis changedPiperOrigin-RevId: 865111649
Library-IDs: google-cloud-gke-hub
Source-link: googleapis/googleapis@7a570661
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
docs: A comment for enum value
ACTIVEin enumStateis changedPiperOrigin-RevId: 865111649
Library-IDs: google-cloud-gke-hub
Source-link: googleapis/googleapis@7a570661
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
docs: A comment for enum value
DISABLINGin enumStateis changedPiperOrigin-RevId: 865111649
Library-IDs: google-cloud-gke-hub
Source-link: googleapis/googleapis@7a570661
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
docs: A comment for field
labelsin message.google.cloud.gkehub.v1.Featureis changedPiperOrigin-RevId: 865111649
Library-IDs: google-cloud-gke-hub
Source-link: googleapis/googleapis@7a570661
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
docs: A comment for field
statein message.google.cloud.gkehub.v1.CommonFeatureStateis changedPiperOrigin-RevId: 865111649
Library-IDs: google-cloud-gke-hub
Source-link: googleapis/googleapis@7a570661
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
docs: A comment for enum value
ENABLINGin enumStateis changedPiperOrigin-RevId: 865111649
Library-IDs: google-cloud-gke-hub
Source-link: googleapis/googleapis@7a570661
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
docs: A comment for message
CommonFeatureStateis changedPiperOrigin-RevId: 865111649
Library-IDs: google-cloud-gke-hub
Source-link: googleapis/googleapis@7a570661
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
docs: A comment for field
request_idin message.google.cloud.gkehub.v1.UpdateFeatureRequestis changedPiperOrigin-RevId: 865111649
Library-IDs: google-cloud-gke-hub
Source-link: googleapis/googleapis@7a570661
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
docs: A comment for field
specin message.google.cloud.gkehub.v1.Featureis changedPiperOrigin-RevId: 865111649
Library-IDs: google-cloud-gke-hub
Source-link: googleapis/googleapis@7a570661
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
docs: A comment for field
membership_specsin message.google.cloud.gkehub.v1.Featureis changedPiperOrigin-RevId: 865111649
Library-IDs: google-cloud-gke-hub
Source-link: googleapis/googleapis@7a570661
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
docs: A comment for field
membership_statesin message.google.cloud.gkehub.v1.Featureis changedPiperOrigin-RevId: 865111649
Library-IDs: google-cloud-gke-hub
Source-link: googleapis/googleapis@7a570661
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
docs: A comment for message
FeatureResourceStateis changedPiperOrigin-RevId: 865111649
Library-IDs: google-cloud-gke-hub
Source-link: googleapis/googleapis@7a570661
END_NESTED_COMMIT
END_COMMIT
This pull request is generated with proto changes between
googleapis/googleapis@33225118
(exclusive) and
googleapis/googleapis@7a570661
(inclusive).
Librarian Version: v0.7.0
Language Image: us-central1-docker.pkg.dev/cloud-sdk-librarian-prod/images-prod/python-librarian-generator@sha256:f5426423676c75008c2135037e7b98f78cbb99f78b3c46fe043b6897be92d836
Fixes #15525